Skip to content

Conversation

@billfienberg
Copy link
Contributor

@billfienberg billfienberg commented Jan 8, 2026

Summary

  • Points snapit action to Shopify/snapit@support-oidc-authentication branch to test OIDC support

Why this PR must be merged to test

GitHub Actions issue_comment events (which trigger /snapit) always run workflows from the default branch, not from the PR branch. This means commenting /snapit on this PR will use the workflow from 2025-10, which still points to Shopify/snapit@main.

To test the OIDC-enabled snapit branch, this PR must be merged first.

Risk assessment

Low risk:

  • If OIDC works → success, then we merge the snapit PR and update this repo to point back to @main
  • If OIDC fails → the publish fails (no packages are published incorrectly), and we revert this change

Test plan

  1. Merge this PR
  2. Open a new PR with a changeset
  3. Comment /snapit on that PR to verify OIDC authentication works

References

🤖 Generated with Claude Code

Points to Shopify/snapit@support-oidc-authentication to test OIDC
support before merging the snapit PR.

Related: Shopify/snapit#52

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@billfienberg billfienberg merged commit 5f9baa7 into 2025-10 Jan 8, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants